-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add refresh button on main feed page #53
base: main
Are you sure you want to change the base?
Conversation
Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-desert-07632091e-53.westus2.5.azurestaticapps.net |
One thing I want to double-check before merging is how attention-getting it is, since some users might not want that. Taking a look now |
I'm not sure how to make this show up. (i tried on my mastodon instance, snoot.tube) do you have a screenshot? |
Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-desert-07632091e-53.westus2.5.azurestaticapps.net |
That seems good to me. When the button appears, does that cause the posts to shift down slightly? i think that could be a little annoying. I'm fine as is for now though. |
I pushed a rebase of this to |
i'll need to reimplement this on the new feed engine / associated control, a lot is changed there (but it also hasn't fully landed as a replacement yet) |
This creates an interval on the first page of a feed that checks for new posts, and provides a way to refresh the feed if it detects new posts.
Currently I have it hardcoded to 60s, but my hope is this can be configured in an eventual app settings.